Skip to content

fix(attestation): base64 encode inline helm charts#2690

Closed
Piskoo wants to merge 1 commit intochainloop-dev:mainfrom
Piskoo:pfm-4212
Closed

fix(attestation): base64 encode inline helm charts#2690
Piskoo wants to merge 1 commit intochainloop-dev:mainfrom
Piskoo:pfm-4212

Conversation

@Piskoo
Copy link
Collaborator

@Piskoo Piskoo commented Jan 23, 2026 •
edited
Loading

Helm chart tarballs stored inline within attestation now are encoded with base64, this fixes issue with UTF-8 validation.

Part of the returned attestation

      "casBackend": {
         "casBackendID": "75c78473-902e-4169-b39e-4770d08200c4",
         "casBackendName": "default-inline",
         "fallback": false
      },
      "materials": [
         {
            "annotations": {
               "chainloop.material.cas.inline": true,
               "chainloop.material.name": "material-1769176887389113820",
               "chainloop.material.type": "HELM_CHART"
            },
            "content": "H4sIFAAAAAAA/ykAK2FIUjBjSE02THk5NWIzVjBkU ... # base64 encoded helm chart",
            "digest": {
               "sha256": "95c7b1d8b26d5f8f59ad5d1a142b6f6b8603ffc9f0f2d9647ff75f8c025f129c"
            },
            "name": "chainloop-1.324.0.tgz"
         }
      ],
      "metadata": {
         "contractName": "myproject-helmcharttest",
         "contractVersion": "2",
         "finishedAt": "2026-01-23T14:01:38.991811223Z",
         "initializedAt": "2026-01-23T14:01:17.906261472Z",
         "name": "helmcharttest",
         "organization": "myorg",
         "project": "myproject",
         "projectVersion": "v1.12.5+next",
         "projectVersionPrerelease": true,
         "team": "",
         "workflowID": "c2b2d61e-d23f-4ea0-8406-2a1ae388e0aa",
         "workflowName": "helmcharttest",
         "workflowRunID": "f0b6f85b-7e12-4102-ae43-7993356bb968"
      },

Closes #2685

@Piskoo
base64 encode helm chart tarball
0a903f4 
Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
@Piskoo Piskoo marked this pull request as ready for review January 23, 2026 14:18
@migmartri
Copy link
Member

migmartri commented Jan 23, 2026

Thanks but I think this is an issue that affects inline items in general not just Helm Charts #2065

@Piskoo
Copy link
Collaborator Author

Piskoo commented Jan 26, 2026

Thanks but I think this is an issue that affects inline items in general not just Helm Charts #2065

Correct, but it is a small change that would enable it for Helm Charts, without breaking anything like using byte instead of string as a type. I'm gonna close it for now.

@Piskoo Piskoo closed this Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@migmartri migmartri Awaiting requested review from migmartri

@jiparis jiparis Awaiting requested review from jiparis

@javirln javirln Awaiting requested review from javirln

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Marshaling error when getting attestations with inline HELM_CHART materials

2 participants

@Piskoo @migmartri